Open
Conversation
Add the US2-FED (IL5) GovCloud data center as region code gov2 across site configuration and content. Updates regions.config.js with derived endpoints, params.yaml allowedRegions and unsupported_sites, the site access table in getting_started, and region-conditional blocks across content/en. Splits blocks containing hardcoded US1-FED values (SDK site enums, URLs, site strings) into dedicated gov2 blocks with US2-FED equivalents. Flags FIPS Agent support for US2-FED in the FIPS compliance page and marks the legacy FIPS Proxy as US2-FED-unsupported. Private-link endpoints, AWS VPCE service names, the oncall domain color, and the rate-limit dashboard IDs are left as TODO pending values from the infra team. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
github-actions
Bot
added
Architecture
Contributor
Government sites share the US1 oncall domain (navy) and do not support AWS PrivateLink. Replace the US2-FED TODOs with the correct gov equivalents: navy oncall, N/A for aws_private_link_cross_region, "not supported" for all aws_private_link_*_service_name keys, and the regular endpoint as the fallback for *_endpoint_private_link keys (matching the US1-FED pattern). Also apply gov2 region handling to the Cdocs .mdoc.md sources (and related partials under layouts/shortcodes/mdoc/en/sdk/) that feed the gitignored SDK and RUM setup pages, so the rebuilt .md artifacts pick up the new region. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Dashboard integration IDs are site-specific and must be looked up in each web app. Rather than ship a broken TODO link, omit the US2-FED entry entirely (matching the existing precedent where AP2 is not listed). A follow-up can add the gov2 entry once the US2-FED dashboard ID is known. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Lost during a merge conflict resolution. The AI summaries and smart chapters banner uses only region-param shortcodes, so it's safe to keep gov and gov2 in the same comma list. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
A few content updates slipped through the initial US2-FED sweep: - Two MCP Server pages added by a recent PR had "not supported" banners scoped to gov only; extended to gov,gov2 since both are listed as unsupported in params.yaml. - A PowerShell metrics example, a react-native codepush comment, and an Android SDK site parameter table row listed US1-FED but not US2-FED. - The getting-started/site page's "select US1-FED" prompt now mentions both gov sites. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
content/en/dd_e2e/ holds Playwright e2e test fixtures (draft: true). These shouldn't be updated as part of DC rollouts - the test expects stable content. Skill and inventory script updated in a separate commit to skip this directory automatically. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
hestonhoffman
changed the title
Row had app.ddog-gov.com (copied from US1-FED); should be us2.ddog-gov.com per the IL5 Deployment Guide's published vanity URL. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
domalessi
approved these changes
Apr 23, 2026
Contributor
domalessi
left a comment
domalessi
left a comment
There was a problem hiding this comment.
Approved for the mdoc files!
jaygorrell
reviewed
Apr 24, 2026
Member
jaygorrell
left a comment
jaygorrell
left a comment
There was a problem hiding this comment.
I'm not familiar with this content but I gave it a pass based on patterns and areas I recongize. I flagged a few things that stood out for attention from someone that knows more about this :)
| eu: 'eu1/v5/datadog-logs.js', | ||
| ap1: 'ap1/v5/datadog-logs.js', | ||
| ap2: 'ap2/v5/datadog-logs.js', | ||
| gov2: 'datadog-logs-v5.js', |
| eu: 'eu1/v6/datadog-rum-slim.js', | ||
| ap1: 'ap1/v6/datadog-rum-slim.js', | ||
| ap2: 'ap2/v6/datadog-rum-slim.js', | ||
| gov2: 'datadog-rum-slim-v6.js', |
Comment on lines
+278
to
279
| gov2: 'us-gov-west-1', | ||
| gov: 'N/A' |
Member
There was a problem hiding this comment.
This is correct, but since gov is N/A I'm just flagging it in case it's not meant to include gov regions here
davidejones
approved these changes
Apr 24, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do? What is the motivation?
Updates docs for US2-FED
Merge instructions
Merge readiness:
Additional notes